Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove extra backslash in unicodedata.st #3336

Closed
wants to merge 1 commit into from

Conversation

jcking
Copy link
Collaborator

@jcking jcking commented Nov 1, 2021

Remove extra backslash in unicodedata.st. specifically \>. Newer versions of StringTemplate will leave the extra backslash in the source file, resulting in an invalid source.

@jcking jcking marked this pull request as ready for review November 1, 2021 19:45
@jcking
Copy link
Collaborator Author

jcking commented Nov 1, 2021

@parrt

@parrt parrt added this to the 4.9.3 milestone Nov 1, 2021
@parrt
Copy link
Member

parrt commented Nov 1, 2021

Hi. I'm checking generated source (updated to 4.3.1 ST4) and it seems ok:

public abstract class UnicodeData {
       private static final Map<String, IntervalSet> propertyCodePointRanges = new HashMap<>(1150);
       private static final Map<String, String> propertyAliases = new HashMap<>(2485);
...

@jcking
Copy link
Collaborator Author

jcking commented Nov 1, 2021

Interesting....let me try to find out why I am getting these dangling backslashes only in our builds.

@jcking
Copy link
Collaborator Author

jcking commented Nov 2, 2021

I think I figured it out. We were importing StringTemplatev4 from the 4.3.1 git tag, which is missing a few fixes one of which is edge case handling of /> per antlr/stringtemplate4#198. I'll close this for now until/if I prove myself otherwise.

@jcking jcking closed this Nov 2, 2021
@parrt
Copy link
Member

parrt commented Nov 2, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants